Skip to content

Comments

Use repository secret instead of default token#63

Merged
tcharding merged 1 commit intorust-bitcoin:masterfrom
nyonson:switch-to-pat
Feb 16, 2026
Merged

Use repository secret instead of default token#63
tcharding merged 1 commit intorust-bitcoin:masterfrom
nyonson:switch-to-pat

Conversation

@nyonson
Copy link
Collaborator

@nyonson nyonson commented Feb 12, 2026

Without setting a token the job uses the default GITHUB_TOKEN of the repository. But this only works if the "Allow GitHub Actions to create and approve pull requests" is enabled on the repository. I don't have perms to see if it is here on rust-psbt, but I bet it isn't. And I bet it is enabled over on rust-bitcoin-maintainer-tools which is why it worked over there.

In any case, it is probably better to keep that setting disabled and use a user account PAT with just repo permission. I added mine to the RBMT_PRS secret in this repository (which I had permissions to do). Updating the job to use it.

I have a PR open on the rbmt side to document these fun little github quirks: rust-bitcoin/rust-bitcoin-maintainer-tools#79

Using nyonson's PAT to trigger the automated toolchain update PRs. This
is slightly preferrable to the default GITHUB_TOKEN.
Copy link
Collaborator

@nymius nymius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for these descriptions, free docs!

ACK 502461d

Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 502461d

@tcharding tcharding merged commit 2113e8d into rust-bitcoin:master Feb 16, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants